Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add system wide condig path #7

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

tclh123
Copy link

@tclh123 tclh123 commented Mar 29, 2018

Use /etc/qcloudcli as config path if it exists and $HOME/.qcloudcli not exists.

支持全局配置目录,对原有 HOME 下配置目录的逻辑没有影响。

@tclh123
Copy link
Author

tclh123 commented Apr 10, 2018

Hello @QcloudApi @zqfan , can you review this?

@QcloudApi
Copy link
Owner

虽然把配置文件放到系统级能够提高便利性,使得其他用户都可以不配置就立即使用,但是也带来了危险,因为配置文件中的密钥对直接关系到现金和重要资产,配置不当权限过大的话容易造成直接的经济损失。所以这个PR可能无法接受。

@tclh123
Copy link
Author

tclh123 commented Apr 11, 2018

Hello @QcloudApi

  1. 提供系统级配置文件的支持,并不会比现在的仅有用户级配置文件支持的情况更不安全。反而更好管理。单个系统级配置文件 sysadmin 可以全权控制,让目标用户组有读权限即可。而若只有用户级配置文件支持,则每个用户必须管理自己的配置文件,sysadmin 难以保证每个用户都配置了正确的文件权限。
  2. 系统级配置文件是一个高级选项,即只在存在系统级配置文件的时候应用,即用户知道自己在做什么。启用系统级配置文件路径并不是默认的,默认还是在用户配置文件路径中初始化配置文件。所以对现有逻辑可以说毫无侵入,只是为高级用户提供了额外支持而已。
  3. 这个 PR 是为了解决我们的使用需求。每个用户必须自己管理配置文件,在我们的使用场景是不安全、不便利、不可行的。如果你有更好的办法,也可以接受。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants